Skip to content

Fix: restore pnpm for changeset publish to apply publishConfig.exports#618

Merged
FL4TLiN3 merged 1 commit intomainfrom
fix/publish-config-exports
Feb 24, 2026
Merged

Fix: restore pnpm for changeset publish to apply publishConfig.exports#618
FL4TLiN3 merged 1 commit intomainfrom
fix/publish-config-exports

Conversation

@FL4TLiN3
Copy link
Contributor

Summary

  • changeset publish only applies publishConfig overrides (like publishConfig.exports) when it detects pnpm and uses pnpm publish
  • The bun migration (PR chore: migrate toolchain to Bun #599) changed packageManager from pnpm@10.10.0 to bun@1.3.5, causing changeset to fall back to npm publish which ignores publishConfig.exports
  • This resulted in published packages having exports: { ".": "./src/index.ts" } instead of the correct dist paths, making them unresolvable by consumers

Changes

  • Restore packageManager to pnpm@10.10.0 (bun ignores this field, so day-to-day dev is unaffected)
  • Restore pnpm-workspace.yaml (turbo needs it when packageManager is pnpm)
  • Add pnpm setup step to release workflow so pnpm publish is available

Test plan

  • bun run typecheck — 23/23 pass
  • bun run format-and-lint — pass
  • bun run test — 19/19 pass
  • bun run build — 23/23 pass
  • bun run check-deps — pass

🤖 Generated with Claude Code

changeset publish only applies publishConfig overrides (like exports) when
using pnpm publish. The bun migration changed packageManager to bun, causing
changeset to fall back to npm publish which ignores publishConfig.exports.

- Add pnpm setup step to release workflow
- Temporarily swap packageManager to pnpm before changeset publish in release
  script so changeset detects pnpm and uses pnpm publish

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FL4TLiN3 FL4TLiN3 force-pushed the fix/publish-config-exports branch from 658e0b4 to 8458c37 Compare February 24, 2026 11:12
@FL4TLiN3 FL4TLiN3 merged commit 12cb3be into main Feb 24, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant